Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feat/142] SocketService 메소드 구현 및 채팅 등록 API 로직 추가 #143

Merged
merged 2 commits into from
Aug 10, 2024

Conversation

Eunjin3395
Copy link
Member

@Eunjin3395 Eunjin3395 commented Aug 10, 2024

🚀 개요

SocketService 메소드 구현 및 채팅 등록 API 로직 추가

🔍 변경사항

  • SOCKET_SERVER_URL 환경변수 추가
  • gradle.yml에 환경변수 추가
  • SocketService 메소드 추가
  • 채팅 등록 서비스 메소드에 로직 추가

⏳ 작업 내용

  • SOCKET_SERVER_URL 환경변수 추가
  • gradle.yml에 환경변수 추가
  • SocketService 메소드 추가
  • 채팅 등록 서비스 메소드에 로직 추가

📝 논의사항

인텔리제이 환경변수에
SOCKET_SERVER_URL / http://localhost:3000 추가 필요합니다!
image.png

@@ -137,7 +138,9 @@ public enum ErrorStatus implements BaseErrorCode {
NOTIFICATION_TYPE_NOT_FOUND(HttpStatus.NOT_FOUND, "NOTI401", "잘못된 알림 타입입니다."),
NOTIFICATION_METHOD_BAD_REQUEST(HttpStatus.BAD_REQUEST, "NOTI402", "알림 생성 메소드 호출이 잘못되었습니다."),

;
// SOCKET 서버 API 호출 에러
SOCKET_API_RESPONSE_ERR(HttpStatus.INTERNAL_SERVER_ERROR, "SOCKET501",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분 SOCKET_API_RESPONSE_ERR -> SOCKET_API_RESPONSE_ERROR 로 변경하셔도 좋을 것 같습니다!

Suggested change
SOCKET_API_RESPONSE_ERR(HttpStatus.INTERNAL_SERVER_ERROR, "SOCKET501",
SOCKET_API_RESPONSE_ERROR(HttpStatus.INTERNAL_SERVER_ERROR, "SOCKET501",

@Eunjin3395 Eunjin3395 merged commit 9dd6965 into develop Aug 10, 2024
@Eunjin3395 Eunjin3395 deleted the feat/142 branch August 10, 2024 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ [Feat] 채팅 등록 API에 socket서버 API 호출 로직 추가
2 participants